Skip to content

Update changelog re 5127#5192

Merged
denik merged 1 commit intomainfrom
denik/changelog-299-1
May 6, 2026
Merged

Update changelog re 5127#5192
denik merged 1 commit intomainfrom
denik/changelog-299-1

Conversation

@denik
Copy link
Copy Markdown
Contributor

@denik denik commented May 6, 2026

No description provided.

@denik denik temporarily deployed to test-trigger-is May 6, 2026 12:05 — with GitHub Actions Inactive
@denik denik temporarily deployed to test-trigger-is May 6, 2026 12:05 — with GitHub Actions Inactive
@denik denik enabled auto-merge May 6, 2026 12:40
@denik denik disabled auto-merge May 6, 2026 13:42
@denik denik merged commit 38eb06e into main May 6, 2026
28 of 30 checks passed
@denik denik deleted the denik/changelog-299-1 branch May 6, 2026 13:42
janniklasrose added a commit that referenced this pull request May 6, 2026
## Summary

- Reverts #5127 (`Persist endpoint UUID for vector_search_endpoints
drift detection`) and the follow-up changelog entry from #5192.
- The badness #5127 was meant to fix — bundle silently rebinding
permissions to a different backing endpoint after an out-of-band
recreate — was actually addressed by the testserver fix in #5186
(`testserver: 404 on permissions GET when V2 parent is gone`). With the
testserver matching real V2 cloud behavior, bundle correctly observes
that the new endpoint has no permissions and creates them, with no
permanent drift afterwards. UUID persistence in state is no longer
necessary.
- Reworks the `drift/recreated_same_name` acceptance test: keeps
endpoint permissions in `databricks.yml`, drops the obsolete "recreate
detected" assertion, and adds a post-deploy `bundle plan` to confirm
there is no permanent drift.

## Test plan

- [x] `./task build` clean.
- [x] `go test ./acceptance -run
'TestAccept/bundle/resources/vector_search_endpoints/drift'` — all green
(terraform + direct).
- [x] `go test ./bundle/direct/dresources/...` — green.
- [x] `./task lint-q` — clean.
- [x] Verified post-deploy plan shows `Plan: 0 to add, 0 to change, 0 to
delete, 2 unchanged` after an out-of-band endpoint recreate, so
permissions don't end up in permanent drift even without UUID-based
recreate detection.

This pull request and its description were written by Isaac.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants